Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 25, 2025

What do these changes do?

This PR fixes a bug where the support group's access rights were not being properly set. The issue occurred because the support group was added to the cache before its access rights were available, and these rights were only provided later when processing organization data.

Bonus:

  • Tags filter always visible in the Tutorials tab, not only when they are grouped by tags

Buggy:
Buggy

Fixed:
Fixed

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Sep 25, 2025
@odeimaiz odeimaiz requested a review from Copilot September 25, 2025 08:19
@odeimaiz odeimaiz added this to the Cheops milestone Sep 25, 2025
@odeimaiz odeimaiz added bug buggy, it does not work as expected a:frontend issue affecting the front-end (area group) labels Sep 25, 2025
@odeimaiz odeimaiz marked this pull request as ready for review September 25, 2025 08:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a bug where the support group's access rights were not being properly set. The issue occurred because the support group was added to the cache before its access rights were available, and these rights were only provided later when processing organization data.

  • Add logic to update support group access rights when processing organization data
  • Check if the support group matches the current organization and update its access rights accordingly

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mergify
Copy link
Contributor

mergify bot commented Sep 25, 2025

🧪 CI Insights

Here's what we observed from your CI run for 36b6bc3.

🟢 All jobs passed!

But CI Insights is watching 👀

@sonarqubecloud
Copy link

@odeimaiz odeimaiz enabled auto-merge (squash) September 25, 2025 08:31
@odeimaiz odeimaiz merged commit 097758b into ITISFoundation:master Sep 25, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group) bug buggy, it does not work as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants